gtk_label_set_attributes() now applies attributes on top of any markup or

	* gtk/gtklabel.c: gtk_label_set_attributes() now applies attributes
	on top of any markup or mnemonic attributes (bug 558409).

	* README: Updated and added release notes for 2.16

svn path=/trunk/; revision=21764

diff --git a/ChangeLog b/ChangeLog
index f947cc48862a8c2dd3e53758f2bc2176bd9a6899..c055372e9f98e00599446c8901f8a78550d43287 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2008-11-06  Tristan Van Berkom <tvb@gnome.org>
+
+       * gtk/gtklabel.c: gtk_label_set_attributes() now applies attributes
+       on top of any markup or mnemonic attributes (bug 558409).
+
+       * README: Updated and added release notes for 2.16
+
 2008-11-06  Richard Hult  <richard@imendio.com>
 
        Bug 558586 – handling of keyboard under darwin (quartz)

diff --git a/README b/README
index 05d4a03e2b07daa1492281970d4a65bc4a224f93..c361be27ab2d3aa7c886e3ff6dc5f38e011c59da 100644 (file)
--- a/README
+++ b/README
@@ -26,6 +26,14 @@ Installation
 
 See the file 'INSTALL'
 
+Release notes for 2.16
+======================
+* The attributes set with gtk_label_set_attributes will now be applied
+  and merged with any other attributes previously effected by way
+  of the GtkLabel:use-underline or GtkLabel:use-markup properties.
+  While it is not recommended to mix markup strings with manually set
+  attributes, if you must; know that the attributes will be applied
+  to the label after the markup string is parsed.
 
 Release notes for 2.14
 ======================

diff --git a/gtk/gtklabel.c b/gtk/gtklabel.c
index bce8a45c50693c462adc9aa22aff2dc025ba590e..e3ecf61115fa3113e9e47c017a081683dade4b54 100644 (file)
--- a/gtk/gtklabel.c
+++ b/gtk/gtklabel.c
@@ -1529,6 +1529,36 @@ gtk_label_set_use_underline_internal (GtkLabel *label,
     }
 }
 
+static void
+gtk_label_compose_effective_attrs (GtkLabel *label)
+{
+  PangoAttrIterator *iter;
+  PangoAttribute    *attr;
+  GSList            *iter_attrs, *l;
+
+  if (label->attrs)
+    {
+      if (label->effective_attrs)
+       {
+         if ((iter = pango_attr_list_get_iterator (label->attrs)))
+           do 
+             {
+               iter_attrs = pango_attr_iterator_get_attrs (iter);
+               for (l = iter_attrs; l; l = l->next)
+                 {
+                   attr = l->data;
+                   pango_attr_list_insert (label->effective_attrs, attr);
+                 }
+               g_slist_free (iter_attrs);
+             }
+           while (pango_attr_iterator_next (iter));
+       }
+      else
+       label->effective_attrs = 
+         pango_attr_list_ref (label->attrs);
+    }
+}
+
 static void
 gtk_label_set_attributes_internal (GtkLabel      *label,
                                   PangoAttrList *attrs)
@@ -1538,17 +1568,8 @@ gtk_label_set_attributes_internal (GtkLabel      *label,
   
   if (label->attrs)
     pango_attr_list_unref (label->attrs);
-
-  if (!label->use_markup && !label->use_underline)
-    {
-      if (attrs)
-       pango_attr_list_ref (attrs);
-      if (label->effective_attrs)
-       pango_attr_list_unref (label->effective_attrs);
-      label->effective_attrs = attrs;
-    }
-
   label->attrs = attrs;
+
   g_object_notify (G_OBJECT (label), "attributes");
 }
 
@@ -1562,11 +1583,17 @@ gtk_label_recalculate (GtkLabel *label)
   guint keyval = label->mnemonic_keyval;
 
   if (label->use_markup)
-    set_markup (label, label->label, label->use_underline);
+    {
+      set_markup (label, label->label, label->use_underline);
+      gtk_label_compose_effective_attrs (label);
+    }
   else
     {
       if (label->use_underline)
-       gtk_label_set_uline_text_internal (label, label->label);
+       {
+         gtk_label_set_uline_text_internal (label, label->label);
+         gtk_label_compose_effective_attrs (label);
+       }
       else
        {
          gtk_label_set_text_internal (label, g_strdup (label->label));
@@ -1624,9 +1651,14 @@ gtk_label_set_text (GtkLabel    *label,
  * @attrs: a #PangoAttrList
  * 
  * Sets a #PangoAttrList; the attributes in the list are applied to the
- * label text. The attributes set with this function will be ignored
- * if the #GtkLabel:use-underline" or #GtkLabel:use-markup properties
- * are set to %TRUE.
+ * label text. 
+ *
+ * <note><para>The attributes set with this function will be applied
+ * and merged with any other attributes previously effected by way
+ * of the #GtkLabel:use-underline or #GtkLabel:use-markup properties.
+ * While it is not recommended to mix markup strings with manually set
+ * attributes, if you must; know that the attributes will be applied
+ * to the label after the markup string is parsed.</para></note>
  **/
 void
 gtk_label_set_attributes (GtkLabel         *label,
@@ -1635,7 +1667,9 @@ gtk_label_set_attributes (GtkLabel         *label,
   g_return_if_fail (GTK_IS_LABEL (label));
 
   gtk_label_set_attributes_internal (label, attrs);
-  
+
+  gtk_label_recalculate (label);
+
   gtk_label_clear_layout (label);  
   gtk_widget_queue_resize (GTK_WIDGET (label));
 }